-
Notifications
You must be signed in to change notification settings - Fork 36
Add PowerVS verify credentials spec #537
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Add PowerVS verify credentials spec #537
Conversation
| - close | ||
| response: | ||
| status: | ||
| code: 34 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have been unable to get a 34 response code when making this API call via curl and the same headers, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I opened a support ticket against the IBM Cloud resource controller service and they were able to find the transaction ID in their logs and verify that the original response code was 200.
I think the evidence points to something in the VCR recording code changing the response code:
- I always get a
200response when making the same API call (path, headers, etc.) usingcurl - I've tested this code live and it has never thrown and exception due to a
34or any other weird response - When recording a new VCR tape of this API call I am always seeing a
34response code.
Will continue to investigate.
.../manageiq/providers/ibm_cloud/power_virtual_servers/cloud_manager/verify_credentials_spec.rb
Outdated
Show resolved
Hide resolved
3c6747c to
bf23dd8
Compare
The default IAM uri for power_virtual_servers is 'https://iam.cloud.ibm.com/identity/token#apikey'. The resource controller URI starts with 'https://resource-controller.cloud.ibm.com' (no 'iaas' subdomain). There's no harm in sanitizing _any_ domain with cloud.ibm in it.
bf23dd8 to
023a569
Compare
|
Checked commits jaywcarman/manageiq-providers-ibm_cloud@e4cbf7b~...023a569 with ruby 3.1.7, rubocop 1.56.3, haml-lint 0.64.0, and yamllint |
When recording new VCR cassettes for #536 I encountered a strange response from the IBM Cloud resource controller API.